Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build with Python 3.13 #256

Closed
wants to merge 1 commit into from
Closed

Fix build with Python 3.13 #256

wants to merge 1 commit into from

Conversation

QuLogic
Copy link
Contributor

@QuLogic QuLogic commented Jun 27, 2024

Python 3.13 has changed how locals() works [1], so this code needs to be a bit more explicit.

[1] https://docs.python.org/3.13/whatsnew/3.13.html#defined-mutation-semantics-for-locals

Python 3.13 has changed how `locals()` works [1], so this code needs to
be a bit more explicit.

[1] https://docs.python.org/3.13/whatsnew/3.13.html#defined-mutation-semantics-for-locals
@bigcat88
Copy link
Owner

bigcat88 commented Jun 28, 2024

I am thinking about changing implementation of this to something like this..:

https://github.com/Visionatrix/Visionatrix/blob/1c38c543d11ffc1e5714af221a42d2ac210f284c/scripts/easy_install.py#L116-L123

edited: I linked an alternative method of obtaining the version, it seems to me that it will be better, do you agree?

@QuLogic
Copy link
Contributor Author

QuLogic commented Jun 28, 2024

I'm sure that's possible, but I went for the minimal change here.

@bigcat88
Copy link
Owner

Anyway thank you for pointing on this.

I am not sure will I publish beta wheels for Python 3.13 in the upcoming release or it will come a month/two later with the drop of manylinux_2014 support and bundling new libehif 1.18.0

Closed in favour of #259

@bigcat88 bigcat88 closed this Jun 28, 2024
@QuLogic QuLogic deleted the py313 branch June 28, 2024 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants